Skip to content

Add 'lock-mode' for Devastator#924

Merged
BenLubar merged 3 commits into
reactivedrop_betafrom
devastator-lock-mode
Aug 1, 2025
Merged

Add 'lock-mode' for Devastator#924
BenLubar merged 3 commits into
reactivedrop_betafrom
devastator-lock-mode

Conversation

@geotavros
Copy link
Copy Markdown
Contributor

@geotavros geotavros commented Jul 27, 2025 •
edited
Loading

This commit adds 2 new ConVars

  • rd_devastator_lockmode_enabled (default is 1) Enables lock-mode secondary attack. Marine is locked in place but weapon fire rate and penetration gets higher.
  • rd_devastator_lockmode_firerate (default is 1.3) Scale factor of the fire rate in lock-mode

When lock-mode is enabled using the ConVar above then the following functionality is enabled for Devastator:

  • Pressing Secondary Fire button locks marine in place
  • Devastator's fire rate is increased by rd_devastator_lockmode_firerate factor.
  • Devastator's pellets will penetrate 1 alien with a chance of 100%
  • Pressing Secondary Fire button again will disable lock-mode
  • Things that disable lock-mode
    • Roll
    • Weapon swap or weapon drop
    • Melee attack
    • Activating jump jet or blink
    • If damaged

@geotavros
Add 'lock-mode' for Devastator
a324845 
This commit adds 2 new ConVars

- rd_devastator_lockmode_enabled (default is 0) Enables lock-mode secondary attack. Marine is locked in place but weapon fire rate and penetration gets higher.
- rd_devastator_lockmode_firerate (default is 1.3) Scale factor of the fire rate in lock-mode

When lock-mode is enabled using the ConVar above then the following functionality is enabled for Devastator:
- Pressing Secondary Fire button locks marine in place
- Devastator's fire rate is increased by rd_devastator_lockmode_firerate factor.
- Devastator's pelets will penetrate 1 alien with a chance of 100%
- Pressing Secondary Fire button again will disable lock-mode
- Things that disable lock-mode
  - Roll
  - Weapon swap or weapon drop
  - Melee attack
  - Activating jump jet or blink
  - If damaged
@geotavros
Add sound name for "lock-mode" activation
ef2aef2 
Used "Burst" type for it because didn't find anything that is suitable.
@jhh8
Copy link
Copy Markdown
Member

jhh8 commented Jul 28, 2025

Why not have it by default

@geotavros
Copy link
Copy Markdown
Contributor Author

geotavros commented Jul 28, 2025

The idea was to playtest it in challenges first to gather players' feedback, fix bugs and improve, for example in Weapons Balancing challenge.

But I'm not against making it enabled by default too.

@geotavros
Copy link
Copy Markdown
Contributor Author

geotavros commented Jul 28, 2025

Players' feedback so far:

  • Add a sound effect when lock-mode gets disabled
  • Add a particle effect when lock-mode is enabled

@GmaOrange
Copy link
Copy Markdown
Contributor

GmaOrange commented Jul 28, 2025

Would be nice to have some kind of visuals (particles\sprites etc) because visual way faster than sound to react from.
Rest seems ok. It doesnt make this gun super usefull still but doesnt ruin either so why not.

@geotavros
Enable 'lock-mode' by default
74eaeda 
This commit sets rd_devastator_lockmode_enabled 1
@BenLubar BenLubar merged commit beb0d60 into reactivedrop_beta Aug 1, 2025
2 checks passed
@BenLubar BenLubar deleted the devastator-lock-mode branch August 1, 2025 15:14
@ax-now
Copy link
Copy Markdown

ax-now commented Aug 6, 2025

Would be cool to have some visual inducator on crosshair when u are in lock-mode.
A visual on the weapon is cool, but generally you look at your sights and what you're shooting at. The marksman has its scope, the medgun has its healing icon. Since the weapon interacts with your movement ability, additional information on the sights would be welcome, what do you think?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@GmaOrange GmaOrange GmaOrange approved these changes

@BenLubar BenLubar Awaiting requested review from BenLubar

@mithrand0 mithrand0 Awaiting requested review from mithrand0

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@geotavros @jhh8 @GmaOrange @ax-now @BenLubar